Skip to content

[Snyk] Fix for 1 vulnerabilities #783

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
officialmofabs wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[Snyk] Fix for 1 vulnerabilities #783

officialmofabs wants to merge 1 commit into from
+2 −3

Conversation

@officialmofabs
Copy link

@officialmofabs officialmofabs commented Dec 21, 2025

snyk-top-banner

Snyk has created this PR to fix 1 vulnerabilities in the maven dependencies of this project.

Snyk changed the following file(s):

  • samples/server/petstore/java-undertow/pom.xml

Vulnerabilities that will be fixed with an upgrade:

Issue Score Upgrade
high severity Allocation of Resources Without Limits or Throttling
SNYK-JAVA-ORGBITBUCKETBC-14465261		   828   com.networknt:audit:
0.1.1 -> 2.1.34
com.networknt:info:
0.1.1 -> 2.1.34
com.networknt:security:
0.1.1 -> 2.1.34
Major version upgrade Proof of Concept

Important

  • Check the changes in this PR to ensure they won't cause issues with your project.
  • Max score is 1000. Note that the real score may have changed since the PR was raised.
  • This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.

For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:

🦉 Allocation of Resources Without Limits or Throttling

@socket-security
Copy link

socket-security bot commented Dec 21, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Addedmaven/​org.springframework/​spring-web@​5.3.33362590100100
Addedmaven/​org.jboss.resteasy/​resteasy-multipart-provider@​4.7.6.Final369990100100
Addedmaven/​org.springframework/​spring-context@​5.3.33369890100100
Addedmaven/​com.google.api-client/​google-api-client@​2.2.04110090100100
Addedmaven/​org.glassfish.jersey.core/​jersey-common@​2.406110010010070
Addedmaven/​org.junit.platform/​junit-platform-runner@​1.10.0931008910070
Addedmaven/​org.glassfish.jersey.connectors/​jersey-apache-connector@​2.37961009010070
Addedmaven/​org.jboss.resteasy/​resteasy-jackson2-provider@​4.7.6.Final98100100100100

View full report

@socket-security
Copy link

socket-security bot commented Dec 21, 2025

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn Critical
Critical CVE: Pivotal Spring Framework contains unsafe Java deserialization methods in maven org.springframework:spring-web

CVE: GHSA-4wrc-f8pq-fpqp Pivotal Spring Framework contains unsafe Java deserialization methods (CRITICAL)

Affected versions: < 6.0.0

Patched version: 6.0.0

From: samples/client/echo_api/java/resttemplate/pom.xml → maven/org.springframework/spring-web@5.3.33

ℹ Read more on: This package | This alert | What is a critical CVE?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Remove or replace dependencies that include known critical CVEs. Consumers can use dependency overrides or npm audit fix --force to remove vulnerable dependencies.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore maven/org.springframework/spring-web@5.3.33. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

@openhands-ai
Copy link

openhands-ai bot commented Dec 21, 2025

Looks like there are a few issues preventing this PR from being merged!

  • GitHub Actions are failing:
    • OpenAPI Generator
    • Linux tests
    • Windows tests
    • .github/workflows/samples-r.yaml

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #783 at branch `snyk-fix-b5fef4113c6447750568d57440b00476`

Feel free to include any additional details that might help me get this PR into a better state.

You can manage your notification settings

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@officialmofabs @snyk-bot